Skip to content

Followup to 6896, add explanation of workaround.#6905

Merged
jimfb merged 1 commit intofacebook:masterfrom
jimfb:document-is-branching
May 27, 2016
Merged

Followup to 6896, add explanation of workaround.#6905
jimfb merged 1 commit intofacebook:masterfrom
jimfb:document-is-branching

Conversation

@jimfb
Copy link
Copy Markdown
Contributor

@jimfb jimfb commented May 27, 2016

Followup to #6896, add explanation of workaround.

@jimfb
Copy link
Copy Markdown
Contributor Author

jimfb commented May 27, 2016

@gaearon

} else if (props.is) {
el = ownerDocument.createElement(this._currentElement.type, props.is);
} else {
// Separate else branch instead of using `props.is || undefined` above becuase of a firefox bug.
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Typo: becuase
Nit: let’s capitalize Firefox :-)

@gaearon
Copy link
Copy Markdown
Collaborator

gaearon commented May 27, 2016

Let’s also add a link to https://bugzilla.mozilla.org/show_bug.cgi?id=1276240

@jimfb jimfb force-pushed the document-is-branching branch from 0ff54ae to 6fbe512 Compare May 27, 2016 16:17
@jimfb jimfb merged commit ba3bfe3 into facebook:master May 27, 2016
@ghost
Copy link
Copy Markdown

ghost commented May 27, 2016

@jimfb updated the pull request.

@zpao zpao added this to the 15-next milestone Jun 1, 2016
zpao pushed a commit to zpao/react that referenced this pull request Jun 8, 2016
zpao pushed a commit that referenced this pull request Jun 14, 2016
@zpao zpao modified the milestones: 15-next, 15.2.0 Jun 14, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants